Skip to content

refactor sending prompt to agent from UI#1539

Merged
adboio merged 1 commit intomainfrom
04-07-refactor_sending_prompt_to_agent_from_ui
Apr 9, 2026
Merged

refactor sending prompt to agent from UI#1539
adboio merged 1 commit intomainfrom
04-07-refactor_sending_prompt_to_agent_from_ui

Conversation

@adboio
Copy link
Copy Markdown
Contributor

@adboio adboio commented Apr 7, 2026

Problem

trying to add more things that prompt the agent automatically, and more annotations to the review panel, was getting messy

Changes

small refactor:

  • adds sendPromptToAgent util that:
    • takes in a prompt+task
    • puts the review panel back to split view if it's open in full view
    • focuses the main session tab
    • sends the prompt to the agent
  • broke out the prompt builders into a new util reviewPrompts.ts
  • made CommentAnnotationself-contained -- now it handles building prompt and sending to agent -- so we don't have to thread a million things through a million components
  • also updated the comment box to match new font styles

i still don't like how task/taskId are getting drilled a million levels deep everywhere, but that happens throughout the whole codebase... i considered a small task context, but it's pretty deeps, i'll revisit that later... for now this just avoids drilling allll the annotation stuff

How did you test this?

manually

@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch from 04dfd84 to 97e2be9 Compare April 7, 2026 20:42
@adboio adboio requested a review from a team April 7, 2026 21:42
@adboio adboio marked this pull request as ready for review April 7, 2026 21:42
@adboio adboio force-pushed the chore/relative-time-utils branch from 3068737 to 906775e Compare April 8, 2026 01:49
@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch 2 times, most recently from 2d9e0a7 to 0607321 Compare April 8, 2026 15:51
@adboio adboio force-pushed the chore/relative-time-utils branch 2 times, most recently from 8020a74 to b07a6ce Compare April 8, 2026 16:01
@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch 3 times, most recently from fe8ee8d to 6050b32 Compare April 8, 2026 22:30
@adboio adboio force-pushed the chore/relative-time-utils branch from b07a6ce to ff7205e Compare April 8, 2026 22:30
@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch from 6050b32 to bf197eb Compare April 8, 2026 22:31
@adboio adboio force-pushed the chore/relative-time-utils branch from ff7205e to e3b240b Compare April 9, 2026 16:11
@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch 2 times, most recently from 162318b to 424d0d2 Compare April 9, 2026 16:18
@adboio adboio force-pushed the chore/relative-time-utils branch from e3b240b to 2cc798d Compare April 9, 2026 16:18
Copy link
Copy Markdown
Contributor Author

adboio commented Apr 9, 2026

Merge activity

  • Apr 9, 5:34 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 9, 5:44 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 9, 5:51 PM UTC: @adboio merged this pull request with Graphite.

@adboio adboio changed the base branch from chore/relative-time-utils to graphite-base/1539 April 9, 2026 17:35
@adboio adboio changed the base branch from graphite-base/1539 to main April 9, 2026 17:42
@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch from 424d0d2 to c52bd8b Compare April 9, 2026 17:43
@adboio adboio merged commit 178803f into main Apr 9, 2026
16 checks passed
@adboio adboio deleted the 04-07-refactor_sending_prompt_to_agent_from_ui branch April 9, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants